Shipping Method Restrictions based on 'StateProvince' #7163
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 2 commits
May 1, 2024 11:07
…handle shipping method restritions based on states/provinces of each country
…to CASCADE NOACTION; because StateProvinceId is on CASCADE DELETE and the CountryId in the StateProvince is on CASCADE DELETE, too.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Almost, all customers say that they need a way to restrict some provinces for some shipping methods which they are not supported by the provider or the store owner would transit orders to those provinces by some another providers! So, I always write my custom codes to all versions to do that! But I decided to add this feature to the core of nop.
All services, controllers, etc. are similar to the 'ShippingMethodRestrictions' based on country.
I create a new entity of 'ShippingMethodStateProvinceMapping' to do that and I put both 'CountryId' and 'StateProvinceId' in that. Some another scenarios could be applied, but this one is more efficient, I think.
Thanks